fix: resolve sidebar clipping and scrolling issues#1010
Conversation
Signed-off-by: yashnagar67 <nitwalababaji@gmail.com>
![gemini-code-assist[bot]](https://avatars.githubusercontent.com/in/956858?s=60&v=4){kind=small}
There was a problem hiding this comment.
Code Review
This pull request modifies the sidebar navigation styling by changing the overflow property to auto and adjusting the maximum height calculation. Feedback indicates that the overflow change in _styles_project.scss requires an explicit max-height to be effective and avoid clipping. Additionally, the update in _sidebar-tree.scss is currently ineffective because the CSS block that applies the height and overflow properties remains commented out.
|
Signed-off-by: yashnagar67 <nitwalababaji@gmail.com>
|
Hi! What's the reason behind moving the file from assets/scss/_sidebar-tree.scss to assets/scss/td/_sidebar-tree.scss? Also, the only change was in the increase of height which allows the current items to not clip anymore but if more items get added onto the sidebar, the sidebar will clip again so the problem wasn't technically solved. |
… and fix sidebar scroll Signed-off-by: yashnagar67 <nitwalababaji@gmail.com>
|
Hello, I worked on another issue and had actually fixed this in the same PR. Unfortunately, I made a mistake while addressing review comments and accidentally reintroduced this issue again. The fix is simple: add |
|
@yashnagar67 Thank you for your contribution! Let's discuss this during the website call tomorrow at 5:30 PM IST | 7 AM CST Add it as an agenda item to the meeting minutes, if you would 🙂 |
|
thanks @Bharath314 for your suggestion, now error is resolved |
This elegantly overrides the upstream Docsy theme constraint, allowing the parent wrapper to handle scrolling natively without needing to duplicate or move upstream SCSS files. Signed-off-by: yashnagar67 <nitwalababaji@gmail.com>
yashnagar67
force-pushed
the
fix-sidebar-clipping
branch
from
f827045 to
1a69648
Compare
|
Thanks @yashnagar67 , 
While reviewing the preview, I noticed the extra sidebar toggle button issue started after moving It may be better to keep |
|
@Sbragul26 thanks i moved sidebar-tree.scss to original location |
Signed-off-by: yashnagar67 <nitwalababaji@gmail.com>
yashnagar67
force-pushed
the
fix-sidebar-clipping
branch
from
1d7d8b1 to
051e280
Compare
|
Thank you for contributing to the Layer5 community! 🎉 \ \ |
6 participants
This fixes the issue where the sidebar navigation fields were being clipped at the bottom and couldn't be fully scrolled.
Fixes: #1009
After experimenting with file overrides, I realized the cleanest fix was to simply add max-height: none to .td-sidebar-nav in our _styles_project.scss.
This directly overrides the aggressive max-height restriction from the upstream Docsy theme. Now, the parent .td-sidebar container handles the scrolling natively, which completely stops the bottom links from getting cut off without requiring us to duplicate or move any theme files.